Skip to content

feat: input pr-number#737

Merged
andykenward merged 5 commits into
mainfrom
pr/number
May 4, 2026
Merged

feat: input pr-number#737
andykenward merged 5 commits into
mainfrom
pr/number

Conversation

@andykenward

@andykenward andykenward commented May 4, 2026

Copy link
Copy Markdown
Owner

Co-authored-by: Copilot copilot@github.com

Pull Request Summary

This PR introduces support for an optional pr-number input so deployments can reliably comment on the correct pull request, especially in fork-safe and workflow-run scenarios. #729

Main Changes

  • Added a new optional action input: pr-number.
  • Wired pr-number into shared input handling and input key constants.
  • Updated pull request comment resolution logic to:
    • Prefer pr-number when provided.
    • Validate input and throw a clear error for invalid values.
    • Keep event-based fallback behavior for pull_request, workflow_dispatch, and workflow_run.
  • Refactored comment resolution with stricter assertions and clearer failure cases.
  • Updated documentation to explain when and why to set pr-number, especially for fork-safe workflow_run usage.
  • Updated deploy workflow behavior to:
    • Trigger on pull requests targeting main.
    • Add a dedicated fork notice job.
    • Skip secret-dependent deploy steps for fork pull requests.
  • Added tests for:
    • Successful PR node ID lookup via pr-number.
    • Invalid pr-number handling.
    • Updated common input shape expectations.
  • Regenerated dist bundles.
  • Added a changeset marking this as a minor feature release.

@changeset-bot

changeset-bot Bot commented May 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b723bfa

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

andykenward and others added 5 commits May 4, 2026 21:34
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
@andykenward andykenward marked this pull request as ready for review May 4, 2026 21:35
@andykenward andykenward changed the title ci: number test feat: input pr-number May 4, 2026
@andykenward andykenward merged commit e0a8f58 into main May 4, 2026
7 checks passed
@andykenward andykenward deleted the pr/number branch May 4, 2026 21:37
@github-actions github-actions Bot mentioned this pull request May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant